Skip to content

feat: presenter#7

Merged
gituser12981u2 merged 12 commits into
alex_stufffrom
feat/presenter
May 24, 2026
Merged

feat: presenter#7
gituser12981u2 merged 12 commits into
alex_stufffrom
feat/presenter

Conversation

@alexcu2718

Copy link
Copy Markdown
Collaborator

Keeping the cmake changes in this as makes it easier to do testing,

Added vk12/vk13 switch, I am pretty sure I could've made it a template class with specialisations for each.
So feel free to rebuke me on that. I don't think it matters(?).

Also added the presenter abstraction.

@gituser12981u2

Copy link
Copy Markdown
Owner

Can you rebase feat/presenter with the newest version of alex_stuff? That should get rid of most of the merge conflicts.

@alexcu2718

Copy link
Copy Markdown
Collaborator Author

Can you rebase feat/presenter with the newest version of alex_stuff? That should get rid of most of the merge conflicts.

Yeah i will tonight. Probably. Maybe

@gituser12981u2 gituser12981u2 changed the title Feat/presenter feat: presenter Mar 22, 2026

@gituser12981u2 gituser12981u2 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FIX PLZ

Comment thread src/backend/vulkan/device/device.cpp Outdated
Comment thread include/quark/vk/presentation/details/swapchain.hpp Outdated
Comment thread src/backend/vulkan/vulkan_context.hpp
Comment thread .gitignore
Comment thread src/backend/vulkan/presentation/presenter_bundle.cpp
Comment thread src/backend/vulkan/vulkan_context.cpp
@gituser12981u2

Copy link
Copy Markdown
Owner

About the Apple issue with VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_VULKAN_1_2_FEATURES and VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_VULKAN_1_3_FEATURES I talked to you about, you know the thing that was giving me a segfault, well I made a fix in the last commit and, for fun, I checked the non Apple path on my MacBook and it somehow worked.

So I guess everything the chain does work and it was some other little problem with the implementation before. I don't know any more. I will just blame MoltenVK. I am making a new commit without the Apple path.

Comment thread src/utils/diagnostics/diagnostic.cpp
Comment thread include/quark/vk/presentation/details/swapchain.hpp
@gituser12981u2 gituser12981u2 dismissed their stale review May 24, 2026 02:28

This review has been satisfied. I don't know why GitHub seems to think it has not been satisfied.

@gituser12981u2 gituser12981u2 merged commit 36487dd into alex_stuff May 24, 2026
5 checks passed
@gituser12981u2 gituser12981u2 deleted the feat/presenter branch May 24, 2026 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants